Skip to content

fix(vc): avoid duplicate output extensions#2796

Closed
wyf027 wants to merge 1 commit into
RVC-Project:mainfrom
wyf027:hermes-auto/rvc-2758-double-extension
Closed

fix(vc): avoid duplicate output extensions#2796
wyf027 wants to merge 1 commit into
RVC-Project:mainfrom
wyf027:hermes-auto/rvc-2758-double-extension

Conversation

@wyf027

@wyf027 wyf027 commented May 17, 2026

Copy link
Copy Markdown

Summary

  • Strip an existing source extension before appending the selected export format in vc_multi
  • Avoid writing files like song.wav.wav when the input file is already a WAV/FLAC/etc.
  • Preserve the original input path for status reporting by using a separate save_path variable for converted formats

Fixes #2758.

Test Plan

  • python3 -m py_compile /tmp/rvc_2758_patchwork/infer/modules/vc/modules.py
  • Local filename behavior check:
    • song.wav + wav -> song.wav
    • song.flac + flac -> song.flac
    • song.mp3 + wav -> song.wav
    • dir/song.wav + mp3 -> song.mp3

Notes

  • This is a minimal WebUI/export path fix and does not touch model inference or algorithm logic.

@wyf027 wyf027 closed this by deleting the head repository Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Proposed fix] A fix for double file extensions

2 participants